Skip to content

KAFKA-20619: ClusterTestExtensions skip jdk.internal.ref.CleanerImpl …#22386

Open
edoardocomar wants to merge 1 commit into
apache:trunkfrom
edoardocomar:ec/KAFKA-20619
Open

KAFKA-20619: ClusterTestExtensions skip jdk.internal.ref.CleanerImpl …#22386
edoardocomar wants to merge 1 commit into
apache:trunkfrom
edoardocomar:ec/KAFKA-20619

Conversation

@edoardocomar
Copy link
Copy Markdown
Contributor

@edoardocomar edoardocomar commented May 27, 2026

…threads

Add "Cleaner-" to the list of Skipped thread prefixes in
ClusterTestExtensions. It's the prefix of daemon threads of
InnocuousThreadGroup used by jdk.internal.ref.CleanerImpl.

Reviewers: Ken Huang s7133700@gmail.com

…threads

Add "Cleaner-" to the list of Skipped thread prefixes in ClusterTestExtensions.
It's the prefix of daemon threads of InnocuousThreadGroup
used by jdk.internal.ref.CleanerImpl.
@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) small Small PRs labels May 27, 2026
@edoardocomar
Copy link
Copy Markdown
Contributor Author

edoardocomar commented May 28, 2026

somewhat related : a while ago I looked into having the list of excluded prefixes extendable
because I use ClusterTest to run tests for kafka plugin implementations (eg authorizers, callbackhandlers, metrics exporters) etc - but that was not generally approved.
#19206

please can any any reviewers of that old PR would take a look at this one ?

https://issues.apache.org/jira/browse/KAFKA-20619

Copy link
Copy Markdown
Collaborator

@m1a2st m1a2st left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@github-actions github-actions Bot removed the triage PRs from the community label May 29, 2026
@edoardocomar
Copy link
Copy Markdown
Contributor Author

edoardocomar commented May 29, 2026

@m1a2st thanks Ken, I thought you were a committer, I still cannot merge :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants